Fix gtk_grid_attach_next_to

The function did not behave as intended when @sibling is NULL.
Also add an example to the docs to clarify the intended behaviour.

gtkrc.key.mac: add Command-cursor text navigation

and some emacs-ish Control bindings that work in native widgets.
Patch from Michael Hutchinson.
(cherry picked from commit ccf12f7b406ecbd8f0c26b0e6dc86d4593144dab)

gtkrc.key.mac: remove half the file, it was an accidential double paste
(cherry picked from commit 79a92f99a9dbdc7b1a651b0e8665807bd89c6632)

Post-release version bump

Halfway to 666 !

Bump version to 3.3.2

Updates

Fix more test breakage

a11y: Hack around infinite loops in parent setting

This is kind of a hack to get rid of infinite loops that occur when
child accessibles try to set their parent upon creation but the parent
accessible creates its children in the initialize vfunc. Because in that
case, the parent will not have an accessible set when the child tries to
access it, because it is still initializing itself. Which will cause a
new accessible to be created.

https://bugzilla.gnome.org/show_bug.cgi?id=660687

Fix make check

Somebody committed a behaviour change that broke his own
tests...tsk, tsk.

Add a css parser test for transparent

printunixdialog: fix a GtkTable->GtkGrid typo

Code was treating the GtkScrolledWindow as a grid, not the grid itself.

Fix handling of GtkCellRendererProgress::inverted

We were setting the wrong field.
https://bugzilla.redhat.com/show_bug.cgi?id=747761

notebook: update tab area allocation when switching page

When the active page is switched, it's important
gtk_notebook_pages_allocate() is called after the bulk of
gtk_notebook_real_switch_page() is run, as the former allocates a
different space according to the currently active tab, which is set by
the latter.
gtk_notebook_pages_allocate() already calls gtk_notebook_redraw_tabs()
when the allocation changes, so just move its call down to
gtk_notebook_real_switch_page() to fix the bug.

https://bugzilla.gnome.org/show_bug.cgi?id=662629

win32: fix unterminated #ifdef error

By removing all ifdefs as gtk/gtkwin32.c is only built
on win32 anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=662481

tooltip: only update shape if the window is realized

If we get a composited-changed signal before the tooltip is ever realized we
have nothing to do.

This actually prevents a segfault in maybe_update_shape() because calling
gtk_widget_get_window() on a non-realized widget returns NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=662467

gtk: move _gtk_modules_has_mixed_deps() to gtkmodlesprivate.h

and remove gtkmainprivate.h completely.

gtk: clean up the private horror

- add gtkmodulesprivate.h and move stuff there from gtkprivate.h
- add gtkprivate.c and move stuff there from gtkmain.c
- add gtkwin32.c and move stuff there from gtkmain.c
- don't redefine GTK_DATADIR and friends in gtkprivate.h
- have _gtk_get_datadir() and friends on all platforms
- remove the horrid hacks where gtkprivate.h can't be included,
  or must be included later due to redefinition of the compile-time
  directories

gtkquartz: don't free the string returned by get_bundle_path()

It's statically allocated.
(cherry picked from commit c040b03c2e32a773a4d9cf4019050c2f8a5b91ce)

quartz: don't return FALSE from GdkDevice::query_state()

Not finding a child window to return is not a failure.

Updated German translation

GtkRange: use the right widget for coordinate translation

GtkRange needs to check if its allocation intersects with the resize
grip allocation (trimming its own allocation if it does).
In order to do that, it needs to translate its allocation into window
coordinates, and before that, find the window to whose the allocation
is relative; code goes all the way finding the right parent widget, but
then doesn't actually use it when translating the coordinates, leading
to using the wrong rectangles for the intersection check.

https://bugzilla.gnome.org/show_bug.cgi?id=662308

Fix document generation in out of tree builds

https://bugzilla.gnome.org/show_bug.cgi?id=661032

Fix natural/min widht calculations for empty grids

https://bugzilla.gnome.org/show_bug.cgi?id=660139

Add GtkSwitch to widget-factory

Distribute tests/css/parser/selector.errors

https://bugzilla.gnome.org/show_bug.cgi?id=662024

Don't refer to --sync anymore

That commandline option got replaced with an environment variable,
so mention that in the warning text.

https://bugzilla.gnome.org/show_bug.cgi?id=662132

DND: Work better with XI2

It turns out that simply using XIGrabKeycode instead of XGrabKey
makes the DND keyboard support mostly work (there seem to be some
minor issues with modifiers). This means we no longer grab the
keyboard actively during DND, which in turn makes Alt-Tab and
other window manager shortcuts work again during DND.

At the same time, bring the DND key handling code into the
multi-backend work, by checking for X11 and XI2 at runtime,
in addition to compile time.

treemodel: document gtk_tree_path_free() as allow-none

https://bugzilla.gnome.org/show_bug.cgi?id=662230

gtk: GtkToolbar returns invalid paths when adding tool items

 * Calculate siblings_path on demand, so that it's always correct
   and doesn't get out of sync with the toolbar state.

https://bugzilla.gnome.org/show_bug.cgi?id=662177

tooltip: add support for opacity and rounded corners for tooltips

Rounded corners now will always work, using XShape in case we're not
running a composite manager.
Also, setting an RGBA visual (if available) on the tooltip toplevel
enables them to be transparent if the theme specifies so.

https://bugzilla.gnome.org/show_bug.cgi?id=599617

GMountOperation on OpenBSD: remove the need for kvm(3)

kvm(3) is considered a deprecated interface, so make
GMountOperation::show-processes use the recommended sysctl(3) interface
instead. This also removes the need to link with libkvm.

https://bugzilla.gnome.org/show_bug.cgi?id=661835

Fixed some strings in Brazilian Portuguese translation

Update VS property sheets

Bid farewell to G_DISABLE_DEPRECATED and the build errors it causes as
warnings are now used to deter people from using deprecated GLib items
in a more subtle manner

Updated Japanese translation

Updated British English translation

Bug 658772: Directory paths for resource directories are hard coded.

Corrected formatting to match coding standards; introduced local statics
to prevent leaking returned strings.

gtk: fix syntax of an annotation

Use G_VALUE_INIT

Instead of an explicit { 0, } when declaring the variable.

gdk: s/gdk_cursor_unref/g_object_unref/

Fixed referenced images for GTK_STOCK_INDENT

https://bugzilla.gnome.org/show_bug.cgi?id=661780

Add deprecation annotations for GtkHandleBox and GtkTable

build: Strip out -DG_DISABLE_DEPRECATED

It's unnecessary pain now that they're warnings.

Bug 660730: Win32: Only use _fstat32 if available

_fstat32 is only introduced with msvcrt80.dll (i.e. Visual C++ 2005), in
which using this function will break compilation with MinGW, which links
against msvcrt.dll.  The msvcrt.lib in the Windows DDK which links to
a later incarnation of the msvcrt.dll in later Windows systems may have
this symbol defined, but that needs to be checked upon to be sure.

Thanks to Dieter Verfaillie for pointing out this problem.

Add deprecation annotations for deprecated functions

We define our own GDK_DEPRECATED[_FOR] macros for this
and allow it to be turned off by defining the
GDK_DISABLE_DEPRECATION_WARNINGS macro.

Include gdk.h in gail.c

This is needed so that we pick up deprecation macros here in the
future.

Add a new symbol

gtk_widget_get_modifier_mask was added recently.

Fix void function n gtkfontbutton.c so it does not try to return a value.
 Fixes bug #661089.

Bug 658767 - Drag and Drop NSEvent capture is racy

Create a synthetic NSMouseLeftDown to store in the GtkQuartzDragSourceInfo
rather than relying on the NSWindow's latest event being the right one (or the
right kind).

Bug 657770 - Write to released memory in gtkdnd-quartz.c

Clear the Drag paste board just before the info->context is released.
This way the GtkDragSourceOwner is released just before the drag context
is and thus can pasteboard:provideDataForType: not accidentally access
an already released drag context

Quartz, Bug 655057: Eliminate Compiler Warning

When compiled with older SDKs, the original change for this bug caused a
compiler warning about NSWindow not being able to handle a setStyleMask
message. This tricks the compiler into thinking that it can.

Add G_GNUC_DEPRECATED annotations in GDK

Bug 658772: Directory paths for resource directories are hard coded.

Provide dynamic path discovery functions as are provided for win32.

quartz: also update window position on windowDidResize

When a window goes fullscreen, the resize also changes its size.

Updated Dutch translation by Wouter Bolsterlee

Updated Dutch translation by Rachid

gtk: remove the private GTK_DEFAULT_ACCEL_MOD_MASK define

and finish the port to using the new modifier abstraction API.
This commit has some evilness, it uses the default display for
the lack of a widget context, and the change to gtkstock.c
is very ugly, but I can't think of anything better given
GtkStockItem needs an accel mask instead of a proper accel
string.

gdk: fix gdk_keyval_to_lower/upper() for Quartz, Win32 and Broadway

In 2.x, the !HAVE_XCONVERTCASE fallback of keyval_convert_case() was
implicitly used as implementation for all !X11 backends.

In 3.x, when this function was virtualized in GdkDisplayManager,
this fallback was moved to the X11 backend and the other backends
"equipped" with /* FIXME implement */ implementations of
keyval_convert_case() which don't convert anything.

Move the fallback code back to gdk/ as default implementation
of GdkDisplayManager::keyval_convert_case() and remove its
implementations is all backends but X11. Also remove the
implementation in Wayland which was a plain copy of what
is now the default implementation.
(cherry picked from commit f46c1b76d839a03bdc8080bc130fabd1068f2557)

gtk: use the new modifier API instead of GTK_DEFAULT_ACCEL_MOD_MASK

in gtkimcontextsimple and gtktreeview, which are the easy places,
but also true bugfixes, because on OSX we can't use the virtual META
here, we needs MOD2 as delivered in key events.

Updated Galician translations

GtkComboBox: Add the format-entry-text signal.

This provides more control over how the selected item is shown in
the entry.
Bug #631167

GtkTreeViewColumn: dispose: Remove from the treeview

This handles the case (with gtkmm) that the GtkTreeViewColumn is
destroyed before the GtkTreeView is destroyed.
Bug #659672

Bug 628936: Minimal change to pass introspection.

gobject_introspection's G-ir-scanner doesn't like the -xobjective-c
option needed to compile ObjectiveC features in quartz implementations
of certain gtk functions. This rearranges the compiler flag environment
variables in Makefile so that G-ir-scanner doesn't see them.

Fixed gtk_cell_area_stop_editing() to properly notify when editing is not canceled

This fixes bug 653289, in gtk+-2 gtk_tree_view_stop_editing() was previously
responsible for notifying with gtk_cell_editable_editing_done(), this detail
was missed when porting the cell renderer management to GtkCellArea.

reftests: add a reftest for the CSS :only-child selector

https://bugzilla.gnome.org/show_bug.cgi?id=660833

all: add an 'only-child' pseudo-class selector

Add an :only-child pseudo-class selector, which selects single children
of a parent container.

https://bugzilla.gnome.org/show_bug.cgi?id=660833

appchooser: make the "Find applications online" follow search state

Set the button as insensitive while searching, so that it can't be
accidentally clicked twice.

https://bugzilla.gnome.org/show_bug.cgi?id=651818

appchooser: add a GCancellable to the search operation

So that we can cancel it when the dialog is destroyed.

https://bugzilla.gnome.org/show_bug.cgi?id=651818

Updated Spanish translation

Fix commit 61d6c73f186a08cd55852c0a44a1704a6fbbbc39

Where the commit suggested "somebody should review it [the test]",
it would have been useful to test this particular commit instead...

Bug 659022 - gtk_tree_model_filter_clear_cache_helper: assertion failed

This bug is resolved by fixing two things in
gtk_tree_model_filter_row_deleted():

  (1) It is possible for an elt to have elt->visible_siter == NULL, when
  it is deleted.  Only call g_sequence_remove() if this pointer is
  non-NULL.

  (2) For the case len (level->seq) > 1, free the elt->children level
  if non-NULL.  Failing to do this means the level will stick around.
  If this child level was not referenced, it will still have a zero
  ref count on its parent which cannot be removed!

For both bugs unit tests have been added in the preceding commit.

Add two more unit tests for bug 659022

Both pointing out problems in gtk_tree_model_filter_row_deleted().

Fix first part of bug 659022

Push creation of path into if clause, the path cannot be created anyway
(and would be meaningless otherwise) if the parent is not visible.

Add unit test for part 1 of bug 659022

This tests a case where gtk_tree_model_filter_row_changed() is run
for a node with visible_siter != NULL, however its parent has
visible_iter == NULL. For this case, no valid paths can be generated
as the node cannot be accessed (because the parent is invisible).

box: add some missing includes

iconcache: Fix gcc warning

filesystemmodel: Add a missing check

Paths with depth > 1 should return FALSE instead of pretending the depth
is 1.

Bug 660730: Use GStatBuf for portability

Thanks to Kean Johnston for pointing this out.

There are a few places in GTK that use "struct stat",
and then g_stat(), rather than using GStatBuf.This breaks things on
Windows. Since the size of struct stat can vary depending on other
flags specified, this has the potential to cause overwrites and is
trivial to fix.

Based on patch submitted by Kean Johnston

gdk: Plug leak in gdk_display_pointer_is_grabbed

gdk_x11_device_manager_core_list_devices returns a new allocated
list, which has to be freed.

valgrind output:

==18686== 160,176 (80,088 direct, 80,088 indirect) bytes in 3,337 blocks are definitely lost in loss record 25,347 of 25,378
==18686==    at 0x4C256DD: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==18686==    by 0x6CD7752: g_malloc (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x6CEE2B6: g_slice_alloc (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x6CCB37D: g_list_prepend (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x654CADA: gdk_x11_device_manager_core_list_devices (gdkdevicemanager-core-x11.c:836)
==18686==    by 0x6531489: gdk_display_pointer_is_grabbed (gdkdisplay.c:1270)
==18686==    by 0x5162E1E: filter_func (ui.c:140)
==18686==    by 0x6558B50: gdk_event_apply_filters (gdkeventsource.c:83)
==18686==    by 0x6558CB3: _gdk_x11_display_queue_events (gdkeventsource.c:197)
==18686==    by 0x6530680: gdk_display_get_event (gdkdisplay.c:311)
==18686==    by 0x65589F1: gdk_event_source_dispatch (gdkeventsource.c:356)
==18686==    by 0x6CD0A0E: g_main_context_dispatch (in /lib64/libglib-2.0.so.0.3000.0)

https://bugzilla.gnome.org/show_bug.cgi?id=660676

Updated Esperanto translation

__SIZEOF_INT__ and __SIZEOF_POINTER__ may not be defined.

Protect __SIZEOF_INT__ and __SIZEOF_POINTER__ with an ifdef to prevent
warnings about them not being defined when including gtktextattributes.h.

https://bugzilla.gnome.org/show_bug.cgi?id=660619

gtkbox: Move private functions to private header

Add 'Since 3.4' indices to the docs

Add new api to gdk.symbols

This keeps 'make check' happy.
And if 'make check' is happy, so am I.

gdkcreen-x11: Use GdkRGBA instead GdkColor

Updated Vietnamese translation

po/vi: import from Damned Lies

uimanager: Deprecate add-tearoffs property

Don't use it. Please.

menu: Deprecate GtkTearoffMenuItem

testgtk: Remove tearoff menus from tests

gtk-demo: Don't use tearoff menus

Demoing bad things, sheesh.

gtkwindow: Clean up gtk_window_map()

Remove unneeded variable and delete trailing whitespace.

docs: Delete duplicate line in gdkwindow.c

handlebox: Deprecate

testgtk: Remove handlebox example

Rather than keeping it and enabling deprecated code in testgtk I decided
to delete the example. The warnings from testgtk about deprecated code
are too useful IMO.
Should we need it again, we can make it into its own test, or add a
testgtkdeprecated.

tests: Remove handleboxes from testmerge example

a11y: Don't include gtkx when not needed

a11y: Remove useless check

Updated Hebrew translation.

tests: Fix a bunch of set-but-unused warnings in the filtermodel test

The whole test doesn't seem to unref any memoery, somebody should review
it.

docs: Fix typo in treeview docs